Hardening: payout transfer preflight validation (no 500s)#126
Hardening: payout transfer preflight validation (no 500s)#126David-code-tang wants to merge 1 commit intoScottcjn:mainfrom
Conversation
|
Strong patch set and the preflight tests pass locally. One blocker before merge: Deployment compatibility: this PR imports Please switch to a deployment-safe import pattern, e.g. try local-module import first (or vice versa): try:
from payout_preflight import ...
except ImportError:
from node.payout_preflight import ...(or equivalent robust fallback). After that I can fast-track merge. |
|
Status: this is functionally superseded by PR #127 (same preflight layer + stronger 2-phase commit fix). Also still has deployment-compat import issue ( Recommendation:
|
|
Closing as superseded by merged PR #127 (includes the same preflight layer + additional 2-phase commit hardening for signed transfers). |
Implements Wave2 bounty #130: adds dependency-light preflight validation for /wallet/transfer and /wallet/transfer/signed, hardens /wallet/transfer to avoid 500s on malformed/non-finite inputs, includes CLI checker and docs.\n\nTests:\n- PYTHONPATH=. python3 -m unittest node/tests/test_payout_preflight.py -v\n\nDocs:\n- docs/PAYOUT_PREFLIGHT.md\n